multiselection: Don't emit nonsensical signals

I forgot to add a check here. We only want to emit
a ::selection-changed if we actually changed it.

NEWS: Updates

gtk: Documentation fixups

Document missing symbols, and other cleanups.

gdk: Documentation fixups

Document some missing symbols.

gsk: Documentation fixups

Document a few missing symbols.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2050

Merge branch 'wip/ebassi/a11y-2' into 'master'

Accessibility cleanup

See merge request GNOME/gtk!2049

NEWS: Updates

gtk-demo: Another way to trigger theme benchmark

The click on the headerbar seems broken atm, so add
an explicit button to trigger the cycling among the
themes.

spinbuttonaccessible: Fix an oversight

We need to pass the accessible to the signal handler.

Trivial typo fix

a11y: Drop GtkWidgetAccessibleClass.notify_gtk

It's finally unused.

Accessible types should either watch properties they are interested in
directly, or should have (private) API to allow widgets to update the
accessible state directly.

a11y: Simplify GtkEntryAccessible

Now that we don't have any additional subclasses of GtkEntryAccessible
in GTK, we can drop all the conditional fluff in the base class.

We still need to subscribe to the global notify signal, because of the
sheer amount of properties watched by GtkEntryAccessible.

a11y: Implement GtkPasswordEntryAccessible properly

GtkPasswordEntryAccessible is not a GtkEntryAccessible any more, so it
will need a proper implementation of various interfaces and
functionality in order to work like any other entry.

a11y: Make GtkSearchEntryAccessible a full GtkWidgetAccessible

We should not inherit from GtkEntryAccessible, given that GtkSearchEntry
is *not* a GtkEntry.

a11y: Simplify GtkScrollbarAccessible

a11y: Simplify GtkScaleButtonAccessible

a11y: Simplify GtkSpinButtonAccessible

We're already listening to the adjustment property on the spin button,
there's no need to reset the adjustment on widget set/unset, since the
accessible instance is always tied to the same widget.

a11y: Simplify GtkTextAccessible

a11y: Simplify GtkTextViewAccessible

Drop the GtkWidgetAccessible.notify_gtk override.

a11y: Simplify GtkColorSwatchAccessible

Drop the GtkWidgetAccessible.notify_gtk override.

a11y: Simplify GtkNotebookAccessible

Let the GtkNotebook explicitly update the accessible object that the
current page has changed, instead of overriding
GtkWidgetAccessible.notify_gtk.

a11y: Simplify GtkWindowAccessible

Have GtkWindow notify of title changes, instead of overriding
GtkWidgetAccessible.notify_gtk.

a11y: Simplify GtkExpanderAccessible

Drop the GtkWidgetAccessible.notify_gtk and AtkObject.initialize
overrides, and have GtkExpander update the state of the accessible
object directly.

a11y: Simplify GtkLevelBarAccessible

Drop the GtkWidgetAccessible.notify_gtk and AtkObject.initialize
overrides, and have GtkLevelBar notify the accessible object of value
changes.

a11y: Split GtkSearchEntryAccessible from GtkEntryAccessible

Use a separate accessible object for the GtkSearchEntry, just like we
did for GtkPasswordEntry.

a11y: Split password entry accessible from GtkEntryAccessible

Use a separate accessible object.

a11y: Simplify GtkProgressBarAccessible

Drop the GtkWidgetAccessibleClass.notify_gtk and the
AtkObjectClass.initialize overrides: they don't do anything relevant.

Instead, have GtkProgressBar update the accessible state when the
fraction changes.

a11y: Use G_DECLARE macro for GtkWidgetAccessible

Allows us to drop the Private structure typedef.

a11y: Update the accessible state when GtkIconView:model changes

Do not use a generic "notify" signal handler.

Additionally, clean up the GtkIconViewAccessible implementation to bring
it up with modern idiomatic GObject.

a11y: Drop notify_gtk() from GtkRangeAccessible

Let the GtkRange notify the accessible instance when the adjustment
changes.

a11y: Drop notify_gtk() from GtkButtonAccessible

Make GtkButton notify the accessible state directly when updating the
label.

a11y: Remove GtkToggleButtonAccessible notify_gtk implementation

The base GtkWidgetAccessible class does the right thing by default.

a11y: Update accessible state for GtkTreeView directly

Do not use GtkWidgetAccessibleClass.notify_gtk().

Notify accessibility state changes from GtkWidget

Piggybacking on blanket GObject::notify is not very efficient.

Move orientable style classes into GtkWidget

It feels slightly wrong to have GtkOrientable operate on widgets, but at
least what happens when an orientable widget changes orientation should
be part of GtkWidget.

This will allow to add more state changes without accessing widget state
from the outside of gtkwidget.c.

Notify mapped state to the accessible object directly

Do not use signal connections: if we have an accessible object, notify
it directly when getting mapped and unmapped.

Drop unused AtkRegistry path

We expect widgets to use their own derived GtkWidgetAccessible type,
these days, and given that we hard code the default accessible type of a
GtkWidget to GtkWidgetAccessible, and that we enforce the dependency of
the type passed to gtk_widget_class_set_accessible_type(), the registry
code path is clearly unused.

Make tooltip properties idiomatic

The tooltip handling in GtkWidget is "special":

 - the string is stored inside the qdata instead of the private
   instance data
 - the accessors call g_object_set() and g_object_get(), and the
   logic is all inside the property implementation, instead of
   being the other way around
 - the getters return a copy of the string
 - the setters don't really notify all the involved properties

The GtkWidgetAccessible uses the (escaped) tooltip text as a source for
the accessible object description, which means it has to store the
tooltip inside the object qdata, and update its copy at construction and
property notification time.

We can simplify this whole circus by making the tooltip properties (text
and markup) more idiomatic:

 - notify all side-effect properties
 - return a constant string from the getter
 - if tooltip-text is set:
   - store the text as is
   - escape the markup and store it separately for the markup getter
 - if tooltip-markup is set:
   - store the markup as is
   - parse the markup and store it separately for the text getter

The part of the testtooltips interactive test that checks that the
getters are doing the right thing is now part of the gtk testsuite, so
we ensure we don't regress in behaviour.

Use modern GObject macros for GtkAccessible

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2048

multiselection: Fix the select_range implementation

When exclusive is TRUE, we would not always emit a
::selection-changed signal that covers all the items
that were unselected.

This commit includes a test.

set: Fix gtk_set_get_max

This function had an off-by-one error. Thankfully,
our selection tests are thorough enough to catch it.

dropdown: Connect a forgotten signal handler

The code was exporting a search_stop function, but
the ui file didn't connect it to the ::stop-search
signal. Fix that.

Merge branch 'frame-clock-start-time' into 'master'

Tweaks to frame clock for better ties to vsync

See merge request GNOME/gtk!2043

Merge branch 'master.msvc' into 'master'

Fix latest GTK master on Visual Studio

See merge request GNOME/gtk!2028

Merge branch 'columnview-reorder-cells' into 'master'

Columnview reorder cells

See merge request GNOME/gtk!2047

Merge branch 'rubberband-again' into 'master'

Rubberband again

See merge request GNOME/gtk!2041

multiselection: Fix the select_range implementation

When exclusive is TRUE, we would not always emit a
::selection-changed signal that covers all the items
that were unselected.

This commit includes a test.

set: Add a few useful functions

Allow to find out if a set is empty, and its
min and max.

columnview: Update column positions on reordering

Tell reordered columns to reorder their cells to
the new position. This is necessary to get things
like separators right. The visible symptom of this
problem was the lack of the right border when the
last column is reorder to another position, since
the title widget was still the last in its container,
so :last-child applied.

columnviewcolumn: Add private api to set the position

This is needed to make columns reorder their cells when
their position in the column views list of columns
changes.

listitemwidget: Add api to reorder children

We need this to keep the rows in sync with the columns
when they are reordered.

columnviewlayout: Handle for_size propertly

When we are given a for_size as width for the whole
column view, we need to distribute it over the columns
as gtk_column_view_allocate_columns would, in order
to find out which for_size to give to each cell.

This is a bit recursive, but works. Since we are
doing this recursion for every row, we should consider
adding a cache for those distributed widths.

Merge branch 'wip/damaged-but-no-frost-bite' into 'master'

x11: defer _NET_WM_FRAME_DRAWN update until frame usable by compositor

See merge request GNOME/gtk!1982

Merge branch 'matthiasc/for-master' into 'master'

gtk-demo: Bring back the applauncher demo

See merge request GNOME/gtk!2040

Merge branch 'ebassi/for-master' into 'master'

Add missing annotation

See merge request GNOME/gtk!2046

columnview: Split gtk_column_view_allocate_columns

Split off a helper function that just distributes
the widths, without actually allocating the columns.

This will be used in measuring in the future.

columnview: Take column width into account for measuring

When measuring the height of a cell, we need to give
it it the width of its column, if we expect realistic
results.

x11: Defer _NET_WM_FRAME_DRAWN update until frame usable by compositor

With the vendor provided Nvidia driver there is a small window of time
after drawing to a GL surface before the updates to that surface
can be used by the compositor.

Drawing is already coordinated with the compositor through the frame
synchronization protocol detailed here:

https://fishsoup.net/misc/wm-spec-synchronization.html

Unfortunately, at the moment, GdkX11Surface tells the compositor the
frame is ready immediately after drawing to the surface, not later,
when it's consumable by the compositor.

This commit defers announcing the frame as ready until it's consumable
by the compositor. It does this by listening for the X server to announce
damage events associated with the frame drawing.  It tries to find the
right damage event by waiting until fence placed at buffer swap time
signals.

x11: Factor out some of frame sync code into subroutines

This commit moves some of the end frame sync counter handling
code to subroutines.

It's a minor readability win, but the main motivation is to
make it easier in a subsequent commit to defer updating the
sync counter until a more appropriate time.

x11: Add back support for the damage extension

commit 14bf58ec5dfdf19e3ca603b977626608dafc729b dropped support
for using the DAMAGE extension since there was no code that
needed it.

We're going to need it again, however, to address an NVidia
vendor driver issue.

This commit does the plumbing to add it back.

filechooserwidget: Avoid an uninitialised value

For NULL filters, we were never calling g_list_store_find, so leaving
filter_index undefined.

demos: Avoid shadowing a global from time.h

Sad but true.

filterlistmodel: Fix fallthrough annotations in unreachable code

They are only unreachable in debug builds but still break the build on
clang.

columview: Make cells propagate request mode

I think this is necessary to make wrapping
labels work.

gtk-demo: Improve wrapping in settings demo

The columnview overrides the width of its content,
so if a label is not willing to wrap its content
below a certain width, it will just get cut off.

Avoid that by not setting width-chars on the wrapping
content.

Add missing annotation

columviewlayout: Skip children when we should

Every layout manager should call gtk_widget_should_layout
on their children before measuring or allocating them.
So do it here too.

listview: Add an example to the docs

The example is an excerpt from the applauncher
demo in gtk4-demo.

gtk-demo: Bring back the applauncher demo

Just without the coverflow. It was a well-documented
demo, so it is useful to keep around.

frame clock: schedule in refresh intervals

When the application does not receive "frame drawn" signals we schedule
the clock to run more or less at intervals equal to the last known
refresh interval. In order to minimize clock skew we have to aim for
exact intervals.

frame clock: Better handle non-regular clock cycles

We try to step the frame clock in whole refresh_interval steps, but to
avoid drift and rounding issues we additionally try to converge it to
be synced to the physical vblank (actually the time we get the
frame-drawn message from the compositor, but these are tied together).

However, the convergence to vsync only really makes sense if the new
frame_time actually is tied to the vsync. It may very well be that
some other kind of event (say a network or mouse event) triggered
the redraw, and not a vsync presentation.

We used to assume that all frames that are close in time (< 4 frames
apart) were regular and thus tied to the vsync, but there is really no
guarantee of that. Even non regular times could be rapid.

This commit changes the code to only do the convergence-to-real-time
if the cause of the clock cycle was a thaw (i.e. last frame drawn and
animating). Paint cycles for any other kind of reason are always
scheduled an integer number of frames after the last cycle that was
caused by a thaw.

frame clock: Track if paint is caused by thaw or not

When we get to a paint cycle we now know if this was caused by a
thaw, which typically means last frame was drawn, or some other event.

In the first case the time of the cycle is tied to the vblank in some
sense, and in the others it is essentially random. We can use this
information to compute better frame times. (Will be done in later
commits.)

Merge branch 'cleanup-snapshot-inhibit' into 'master'

reftest: Clean up reftest_inhibit_snapshot()

See merge request GNOME/gtk!2034

Drop gtk_multi_selection-copy

It is not used anymore.

listbase: Redo rubberbanding

Make it so that the selection is only updated in the end.

demos/gtk-demo/listview*.c: Include stdlib.h

atoi() and qsort() both require this header for them to be defined

modules: Fix build on Visual Studio

Visual Studio does not allow decorating functions with '__declspec (dllexport)'
if a prototype exists and is not decorated with '__declspec (dllexport)' as
well, so we cannot just decorate g_io_module_[load|unload|query] in the various
module sources with G_MODULE_EXPORT because the prototypes of these functions
have been marked with _GLIB_EXTERN, which equates to 'extern' unless overridden

Fix this by overriding _GLIB_EXTERN with the appropriate visibility flag, as we
have used to define _GDK_EXTERN.  Unfortunately, we can't just use _GDK_EXTERN
G_MODULE_EXPORT as they may have not been defined yet for our use

Do this across the board for all modules, even if they are not buildable on
Visual Studio nor Windows, for consistency's sake.

Merge branch 'fix-quadratic-add' into 'master'

Avoid quadratic slowdown in gtk_widget_add()

See merge request GNOME/gtk!2027

Add gtk_selection_model_[un]select_callback

Add a methods to add or remove a whole set
(specified via a query-range style callback).

set: Add a few useful functions

Allow to find out if a set is empty, and its
min and max.

Merge branch 'matthiasc/for-master' into 'master'

more documentation for list widgets

See merge request GNOME/gtk!2039

Merge branch 'fribidi-include' into 'master'

wayland: Remove unused fribidi include

See merge request GNOME/gtk!2037

docs: Expand list widget docs

Begin to flesh out the long descriptions for GtkListView,
GtkGridView and GtkColumnView.

wayland: Remove unused fribidi include

The included fribidi header is not used in gdkkeys-wayland.c and already
included in gdk.c which causes linker issues due to the header defining
a global variable.

gtk-demo: Cosmetics

Remove a few instances of double semicolon.

Merge branch 'matthiasc/for-master' into 'master'

gtk-demo: Allow editing in the settings demo

See merge request GNOME/gtk!2033

Merge branch 'list-widget-styles' into 'master'

List widget styles

Closes #2818

See merge request GNOME/gtk!2035

Merge branch 'demo-drop-awards' into 'master'

gtk-demo: Drop the awards

See merge request GNOME/gtk!2036

Avoid quadratic slowdown in gtk_widget_add()

If you add a widget to a parent, this will invalidate the css nodes
for parent/siblings. Afterwards, if the parent is mapped, we will
realize the new child. This calls gtk_widget_update_alpha() which
needs the css opacity, so it revalidates the css.

Thus, for each widget_add (while visible) will trigger a full
revalidation of each sibling. If you add N children to a parent that
leads to O(N^2) revalidations.

To demo this I changed gtk-demo to always double the count
(independent of the fps) and print the time it took. Here is the
results (after a bit):

Setting fishbowl count=256 took 3,4 msec
Setting fishbowl count=512 took 10,1 msec
Setting fishbowl count=1024 took 34,1 msec
Setting fishbowl count=2048 took 126,3 msec
Setting fishbowl count=4096 took 480,3 msec
Setting fishbowl count=8192 took 1892,7 msec
Setting fishbowl count=16384 took 7751,0 msec
Setting fishbowl count=32768 took 38097,7 msec
Setting fishbowl count=65536 took 191987,7 msec

To fix this we drop gtk_widget_update_alpha() and just
calculate it when needed (which is only in a single place).
It was really only necessary because we previously set
the alpha on the surface.

With this fix the above becomes:

Setting fishbowl count=256 took 1,0 msec
Setting fishbowl count=512 took 1,9 msec
Setting fishbowl count=1024 took 3,7 msec
Setting fishbowl count=2048 took 7,4 msec
Setting fishbowl count=4096 took 18,1 msec
Setting fishbowl count=8192 took 31,0 msec
Setting fishbowl count=16384 took 66,3 msec
Setting fishbowl count=32768 took 126,7 msec
Setting fishbowl count=65536 took 244,6 msec
Setting fishbowl count=131072 took 492,2 msec
Setting fishbowl count=262144 took 984,3 msec

gtk-demo: Drop the awards

This was a neat idea, but maintaining it at a sufficient
level is too much work.

list widgets: Document css structure

We didn't fill in this expected part of the widget
documentation yet.

Merge branch 'ensure-style-no-recurse' into 'master'

Avoid recursion in gtk_css_node_ensure_style()

See merge request GNOME/gtk!2031

docs: Fix a parameter name mismatch

Change css names of list widget

The new names are

GtkListView - listview row
GtkGridView - gridview child
GtkColumView - columnview header
               columnview listview row

Adwaita css has been updated to preserve
existing styles.

Fixes: #2818

reftest: Clean up reftest_inhibit_snapshot()

This was done in a weird way where we always call reftest_uninhibit_snapshot()
on paint, and then re-inhibited it if it wasn't inhibited. To make this
work it also started with an extra inhibit.

This is very contorted and based on how this historically worked. This
changes it to just do:

  if (inhibit_count > 0)
    return;

And keep inhibit_count at its initial zero value unless it is actually
inhibited.

Merge branch 'fix-snapshot' into 'master'

snapshot: Fix assert if paint gets scheduled immediately

See merge request GNOME/gtk!2032

gtk-demo: Allow editing in the settings demo

This gives us an example of a columnview with editable
content, which we didn't have so far.

snapshot: Fix assert in inhibition

In https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/2027 i was getting

Bail out! ERROR:../testsuite/reftests/reftest-snapshot.c:212:reftest_uninhibit_snapshot: assertion failed: (inhibit_count > 0)

In (for example the box-shadow-changes-modify-clip reftest. I can reproduce this (on master) with:

```
$ xvfb-run -a -s "-screen 0 1024x768x24" meson test --suite gtk:reftest "reftest box-shadow-changes-modify-clip.ui"
...
1/1 gtk:reftest / reftest box-shadow-changes-modify-clip.ui ERROR          0.77s
``

Fix this by re-inhibiting if we didn't draw anything, or we will get an assert the next paint.